Skip to content

Deprecate XCMv2#4131

Merged
franciscoaguirre merged 28 commits into
masterfrom
deprecate-xcm-v2
May 27, 2024
Merged

Deprecate XCMv2#4131
franciscoaguirre merged 28 commits into
masterfrom
deprecate-xcm-v2

Conversation

@franciscoaguirre
Copy link
Copy Markdown
Contributor

@franciscoaguirre franciscoaguirre commented Apr 15, 2024

Marked XCMv2 as deprecated now that we have XCMv4.
It will be removed sometime around June 2024.

@franciscoaguirre franciscoaguirre marked this pull request as ready for review April 15, 2024 13:26
@franciscoaguirre franciscoaguirre requested a review from a team as a code owner April 15, 2024 13:26
Comment thread polkadot/xcm/src/lib.rs
Comment thread polkadot/xcm/src/lib.rs
Comment thread polkadot/xcm/src/v2/mod.rs Outdated
Comment thread polkadot/xcm/src/v3/mod.rs
@franciscoaguirre franciscoaguirre added the T6-XCM This PR/Issue is related to XCM. label May 23, 2024
Copy link
Copy Markdown
Contributor

@bkontur bkontur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added some suggestions/nits to avoid using concrete XCM versions where not needed, so we don't need to do this again with newer versions

Comment thread polkadot/xcm/pallet-xcm/src/benchmarking.rs Outdated
Comment thread polkadot/xcm/pallet-xcm/src/benchmarking.rs Outdated
Comment thread polkadot/xcm/pallet-xcm/src/benchmarking.rs Outdated
Comment thread cumulus/pallets/xcmp-queue/src/mock.rs Outdated
franciscoaguirre and others added 14 commits May 24, 2024 09:35
…idge-hub-rococo/src/tests/send_xcm.rs

Co-authored-by: Branislav Kontur <bkontur@gmail.com>
…idge-hub-rococo/src/tests/send_xcm.rs

Co-authored-by: Branislav Kontur <bkontur@gmail.com>
…idge-hub-rococo/src/tests/send_xcm.rs

Co-authored-by: Branislav Kontur <bkontur@gmail.com>
…idge-hub-rococo/src/tests/send_xcm.rs

Co-authored-by: Branislav Kontur <bkontur@gmail.com>
…idge-hub-rococo/src/tests/send_xcm.rs

Co-authored-by: Branislav Kontur <bkontur@gmail.com>
…idge-hub-westend/src/tests/send_xcm.rs

Co-authored-by: Branislav Kontur <bkontur@gmail.com>
…idge-hub-westend/src/tests/send_xcm.rs

Co-authored-by: Branislav Kontur <bkontur@gmail.com>
…idge-hub-westend/src/tests/send_xcm.rs

Co-authored-by: Branislav Kontur <bkontur@gmail.com>
…idge-hub-westend/src/tests/send_xcm.rs

Co-authored-by: Branislav Kontur <bkontur@gmail.com>
…idge-hub-westend/src/tests/send_xcm.rs

Co-authored-by: Branislav Kontur <bkontur@gmail.com>
Co-authored-by: Branislav Kontur <bkontur@gmail.com>
Co-authored-by: Branislav Kontur <bkontur@gmail.com>
Co-authored-by: Branislav Kontur <bkontur@gmail.com>
@paritytech-cicd-pr
Copy link
Copy Markdown

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-clippy
Logs: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/6287136

@franciscoaguirre franciscoaguirre added this pull request to the merge queue May 27, 2024
Merged via the queue into master with commit 9201f9a May 27, 2024
@franciscoaguirre franciscoaguirre deleted the deprecate-xcm-v2 branch May 27, 2024 06:36
@Polkadot-Forum
Copy link
Copy Markdown

This pull request has been mentioned on Polkadot Forum. There might be relevant details there:

https://forum.polkadot.network/t/psa-xcmv2-deprecation/8367/1

hitchho pushed a commit to ibp-network/polkadot-sdk that referenced this pull request Jun 5, 2024
Marked XCMv2 as deprecated now that we have XCMv4.
It will be removed sometime around June 2024.

---------

Co-authored-by: Branislav Kontur <bkontur@gmail.com>
TarekkMA pushed a commit to moonbeam-foundation/polkadot-sdk that referenced this pull request Aug 2, 2024
Marked XCMv2 as deprecated now that we have XCMv4.
It will be removed sometime around June 2024.

---------

Co-authored-by: Branislav Kontur <bkontur@gmail.com>
@bkontur bkontur mentioned this pull request Mar 5, 2025
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T6-XCM This PR/Issue is related to XCM.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants